Skip to content

test: use dynamic ports for e2e tests - #23313

Merged
clydin merged 2 commits into
angular:mainfrom
jbedard:dynamic-ports
Jun 10, 2022
Merged

test: use dynamic ports for e2e tests#23313
clydin merged 2 commits into
angular:mainfrom
jbedard:dynamic-ports

Conversation

@jbedard

@jbedard jbedard commented Jun 7, 2022

Copy link
Copy Markdown
Contributor

To allow concurrent tests on the same machine...

Can probably just squash the commits? 🤷

@jbedard
jbedard force-pushed the dynamic-ports branch 11 times, most recently from b3b4df4 to c06031e Compare June 9, 2022 00:38
@jbedard
jbedard marked this pull request as ready for review June 9, 2022 01:21
@jbedard
jbedard requested review from alan-agius4 and clydin June 9, 2022 01:21
Comment thread tests/legacy-cli/e2e/tests/i18n/ivy-localize-serviceworker.ts Outdated
@clydin
clydin removed the request for review from alan-agius4 June 10, 2022 13:41
@clydin clydin added action: merge The PR is ready for merge by the caretaker target: minor This PR is targeted for the next minor release labels Jun 10, 2022
@clydin
clydin merged commit 58c2c09 into angular:main Jun 10, 2022
@angular-automatic-lock-bot

Copy link
Copy Markdown

This issue has been automatically locked due to inactivity.
Please file a new issue if you are encountering a similar or related problem.

Read more about our automatic conversation locking policy.

This action has been performed automatically by a bot.

@angular-automatic-lock-bot angular-automatic-lock-bot Bot locked and limited conversation to collaborators Jul 11, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

action: merge The PR is ready for merge by the caretaker target: minor This PR is targeted for the next minor release

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants